Skip to content

Fix workflows#1396

Closed
ncr38 wants to merge 2 commits intoopen-cluster-management-io:mainfrom
ncr38:fix-workflows
Closed

Fix workflows#1396
ncr38 wants to merge 2 commits intoopen-cluster-management-io:mainfrom
ncr38:fix-workflows

Conversation

@ncr38
Copy link

@ncr38 ncr38 commented Feb 23, 2026 •
edited by coderabbitai bot
Loading

Summary

Related issue(s)

Fixes #

Summary by CodeRabbit

Release Notes

  • Chores
    • Added GitHub Actions workflows for continuous integration, production deployment, hotfix deployment, and release management
    • Added Helm chart artifact management workflow for automated pushes to Nexus repository
    • Added CI configuration with build profiles for multiple application types

ncrbeingncr and others added 2 commits February 20, 2026 16:19
@ncrbeingncr @ncr38
Add build and chart push workflows (open-cluster-management-io#5)
1fc5327 
<!--
Thanks for creating a pull request!

If this is your first time, please make sure to review CONTRIBUTING.MD.

Please copy the appropriate `:text:` or icon to the beginning of your PR
title:

:sparkles: ✨ feature
:bug: 🐛 bug fix
:book: 📖 docs
:memo: 📝 proposal
:warning: ⚠️ breaking change
:seedling: 🌱 other/misc
:question: ❓ requires manual review/categorization

-->
## Summary

## Related issue(s)

Fixes #

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Chores**
* Added automated CI/CD pipeline configuration for continuous
integration
* Configured production deployment workflow with support for multiple
build profiles
* Implemented automated Helm chart validation, packaging, and artifact
repository integration

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Co-authored-by: Navin Chandra Rai <navinchandrarai444@gmail.com>
@ncr38
Add deploy to release hotfix
7b03da9
@openshift-ci openshift-ci bot requested review from deads2k and jnpacker February 23, 2026 13:03
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 23, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ncr38
Once this PR has been reviewed and has the lgtm label, please assign deads2k for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@coderabbitai
Copy link

coderabbitai bot commented Feb 23, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 80ac13c and 7b03da9.

📒 Files selected for processing (5)
  • .github/workflows/build.yaml
  • .github/workflows/deploy_to_prod.yaml
  • .github/workflows/deploy_to_release_hotfix.yaml
  • .github/workflows/helm_chart_push_nexus.yaml
  • ci.yaml

Walkthrough

This PR introduces GitHub Actions CI/CD workflows and configuration for automated build, deployment, and Helm artifact management. Five new files are added: CI configuration specifying Docker-based build profiles, a reusable build workflow, production and hotfix deployment workflows, and a comprehensive Helm chart push workflow with Nexus integration.

Changes

Cohort / File(s) Summary
CI Configuration
ci.yaml		 Defines project metadata and multiple Docker-based build/deploy profiles including registration, work, placement, registration-operator, and addon-manager with corresponding Dockerfile and registry paths.
Build and Deployment Workflows
.github/workflows/build.yaml, .github/workflows/deploy_to_prod.yaml, .github/workflows/deploy_to_release_hotfix.yaml		 Three new workflows using reusable templates from Groww organization. Build workflow triggers on PR and issue comments; prod deployment and hotfix workflows trigger via manual dispatch with configurable build profiles and versions.
Helm Artifact Management
.github/workflows/helm_chart_push_nexus.yaml		 Complex workflow orchestrating Helm chart packaging and Nexus artifact push with RC version detection, branch validation, optional manual approval, Git tagging, GitHub release creation, and dry-run support. Includes multiple conditional steps and error handling.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ncr38 ncr38 closed this Feb 23, 2026
@codecov
Copy link

codecov bot commented Feb 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.69%. Comparing base (80ac13c) to head (7b03da9).

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #1396   +/-   ##
=======================================
  Coverage   61.69%   61.69%           
=======================================
  Files         226      226           
  Lines       18614    18614           
=======================================
  Hits        11484    11484           
  Misses       5917     5917           
  Partials     1213     1213
Flag Coverage Δ
unit 61.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@deads2k deads2k Awaiting requested review from deads2k

@jnpacker jnpacker Awaiting requested review from jnpacker

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ncr38 @ncrbeingncr